Skip to content
This repository was archived by the owner on Dec 21, 2021. It is now read-only.

Adds monitoring of systemd units#53

Merged
soenkeliebau merged 4 commits intostackabletech:mainfrom
soenkeliebau:systemd_monitoring
Feb 18, 2021
Merged

Adds monitoring of systemd units#53
soenkeliebau merged 4 commits intostackabletech:mainfrom
soenkeliebau:systemd_monitoring

Conversation

@soenkeliebau
Copy link
Copy Markdown
Member

@soenkeliebau soenkeliebau commented Feb 16, 2021

During startup, every unit will be checked once every second for 10 seconds, after this it will be considered as "running".

When a unit is considered as "running" the state will be checked once every 10 seconds.

This is a simpler version of what we want to provide in a final implementation, where we listen to changes on dbus instead of polling periodically.

fixes #10

During startup, every unit will be checked once every second for 10 seconds, after this it will be considered as "running".

When a unit is considered as "running" the state will be checked once every 10 seconds.

This is a simpler version of what we want to provide in a final implementation, where we listen to changes on dbus instead of polling periodically.
@soenkeliebau soenkeliebau requested a review from a team February 16, 2021 15:28
Copy link
Copy Markdown
Member

@lfrancke lfrancke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

minor stuff

Comment thread src/provider/states/running.rs Outdated
Comment thread src/provider/states/running.rs
Comment thread src/provider/states/starting.rs
Comment thread src/provider/states/starting.rs
Comment thread src/provider/systemdmanager/manager.rs Outdated
Comment thread src/provider/systemdmanager/manager.rs
@soenkeliebau soenkeliebau merged commit 21ecca5 into stackabletech:main Feb 18, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement systemd unit monitoring

2 participants